Skip to content

Elevation rate limit#45

Draft
morganchristiansson wants to merge 1 commit into
Return-To-The-Roots:masterfrom
morganchristiansson:elevation-sensitivity
Draft

Elevation rate limit#45
morganchristiansson wants to merge 1 commit into
Return-To-The-Roots:masterfrom
morganchristiansson:elevation-sensitivity

Conversation

@morganchristiansson

Copy link
Copy Markdown
Contributor

Mapeditor on Discord reports:

2.) The elevation adjustment is too sensitive. Making finer elevation corrections to a map requires a lot of patience and skillful handling of the corresponding mouse settings – the elevations change too quickly and too drastically.

Add adjustable rate limit based on time (not fps).
image

@morganchristiansson

Copy link
Copy Markdown
Contributor Author

I will have another look at diff with fresh eyes. Maybe some things can be cleaner.
But I already spent a lot of time cleaning it.
All review comments welcome.

@morganchristiansson

Copy link
Copy Markdown
Contributor Author

Actually I'm not even sure we need adjustable rate.
Need feedback from Mapeditor (discord user), maybe just a fixed value is good. Or maybe something slightly different is needed.

I'm a bit too eager to submit PRs maybe lol. I will change it to draft pending feedback.

@morganchristiansson morganchristiansson marked this pull request as draft June 22, 2026 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant